Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Issue #10279 - Update to setuid 2.0.0-SNAPSHOT #10280

Closed
wants to merge 2 commits into from
Closed

Issue #10279 - Update to setuid 2.0.0-SNAPSHOT #10280

wants to merge 2 commits into from

Conversation

joakime
Copy link
Contributor

@joakime joakime commented Aug 9, 2023 •
edited
Loading

Update to use 2.0.0-SNAPSHOT versions of jetty-setuid.

Disabled setuid-osx, as we don't have a machine to build the intel and arch versions.

@joakime joakime added Bug For general bugs on Jetty side dependencies Pull requests that update a dependency file Jetty 12 labels Aug 9, 2023
@joakime joakime added this to the 12.0.x milestone Aug 9, 2023
@joakime joakime self-assigned this Aug 9, 2023
@joakime joakime mentioned this pull request Aug 9, 2023
Closed
@joakime joakime linked an issue Aug 9, 2023 that may be closed by this pull request
Update Jetty lib-setuid for Jetty 12 #10279
Closed
@lorban
Copy link
Contributor

lorban commented Aug 14, 2023

I wonder if it wouldn't be simpler to rewrite setuid with JNA rather than maintaining native builds?

@joakime
Copy link
Contributor Author

joakime commented Aug 14, 2023

I wonder if it wouldn't be simpler to rewrite setuid with JNA rather than maintaining native builds?

Go for it, create a new repo on https://github.com/jetty-project/ for it.

@lorban
Copy link
Contributor

lorban commented Aug 15, 2023

@joakime I created a jetty.toolchain branch instead: jetty/jetty-toolchain#278

@joakime
Copy link
Contributor Author

joakime commented Aug 28, 2023

This PR is out of date, jetty-setuid-jna 2.0.1 is now merged.

@joakime joakime closed this Aug 28, 2023
@joakime joakime deleted the fix/12.0.x/setuid-2 branch August 28, 2023 02:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@olamy olamy Awaiting requested review from olamy

@jmcc0nn3ll jmcc0nn3ll Awaiting requested review from jmcc0nn3ll

@lorban lorban Awaiting requested review from lorban

Assignees

@joakime joakime

Labels
Bug For general bugs on Jetty side dependencies Pull requests that update a dependency file
Projects
None yet
Milestone
12.0.x
Development

Successfully merging this pull request may close these issues.

Update Jetty lib-setuid for Jetty 12
2 participants
@joakime @lorban